Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

7 #183

Closed
wants to merge 78 commits into from
Closed

7 #183

wants to merge 78 commits into from

Conversation

n-ce
Copy link
Owner

@n-ce n-ce commented May 1, 2024

The point of this pr is enable link previews for https://ytify.netlify.app?s=id on the social medias.

The next stage is converting LIT components to Solid-JS. There are advantages to this such as component transparency and reduced complexity in wrapping the components around points of usage. We use Solid-JS simply as a library and not a framework in this case. Our priority is still HTML which is why the toggle-switch will continue to be a web component so that it can be written directly in HTML without any JS involved in injecting it.

The third stage would be involving breaking down the fetchList method and creating collection item components. The advantages to this is only loading collections on demand and onto list instead of their own collapsable children.

Copy link

netlify bot commented May 1, 2024

Deploy Preview for ytify ready!

Name Link
🔨 Latest commit ef45037
🔍 Latest deploy log https://app.netlify.com/sites/ytify/deploys/6655adabc097d40008c3831d
😎 Deploy Preview https://deploy-preview-183--ytify.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.
Lighthouse
Lighthouse
1 paths audited
Performance: 90
Accessibility: 97
Best Practices: 92
SEO: 83
PWA: 100
View the detailed breakdown and full score reports

To edit notification comments on pull requests, go to your Netlify site configuration.

@n-ce n-ce changed the base branch from main to v7 May 28, 2024 15:54
@n-ce n-ce changed the base branch from v7 to main May 28, 2024 15:56
@n-ce n-ce closed this May 28, 2024
@n-ce n-ce deleted the oembed branch May 28, 2024 16:00
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Support Live Streams
3 participants